Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/property search improvements #199

Merged
merged 66 commits into from
May 20, 2024
Merged

Conversation

@aem-code-sync aem-code-sync bot temporarily deployed to feat/property-search-improvements April 1, 2024 21:26 Inactive
@aem-code-sync aem-code-sync bot temporarily deployed to feat/property-search-improvements April 23, 2024 16:54 Inactive
@aem-code-sync aem-code-sync bot temporarily deployed to feat/property-search-improvements April 24, 2024 19:37 Inactive
@aem-code-sync aem-code-sync bot temporarily deployed to feat/property-search-improvements April 29, 2024 21:03 Inactive
@aem-code-sync aem-code-sync bot temporarily deployed to feat/property-search-improvements April 29, 2024 21:03 Inactive
@aem-code-sync aem-code-sync bot temporarily deployed to feat/property-search-improvements April 30, 2024 13:06 Inactive
@aem-code-sync aem-code-sync bot temporarily deployed to feat/property-search-improvements April 30, 2024 13:08 Inactive
@badvision
Copy link
Collaborator

Looks good. Only thing odd I am seeing is the horizontal div/spacer appearing over search results when they are loading. Not a deal-breaker, it just seems out of place.
image

@badvision
Copy link
Collaborator

"Living in" section on communities page has unreadable text
image

rrusher
rrusher previously approved these changes Apr 30, 2024
Copy link
Collaborator

@rrusher rrusher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I appreciate the code coverage on the API. It's also well commented.

While I wouldn't advocate for rewriting innerHTML sections using the dom-helpers before go-live, I do think it would be a great task for the customer to learn how maintain the code by refactoring those sections.

@aem-code-sync aem-code-sync bot temporarily deployed to feat/property-search-improvements May 8, 2024 12:47 Inactive
@bstopp bstopp marked this pull request as ready for review May 8, 2024 12:47
@aem-code-sync aem-code-sync bot temporarily deployed to feat/property-search-improvements May 8, 2024 19:17 Inactive
@bstopp bstopp merged commit 9304fb1 into main May 20, 2024
1 of 2 checks passed
@bstopp bstopp deleted the feat/property-search-improvements branch May 20, 2024 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants